Skip to content

Update go runtime 1.24.7#1425

Closed
mrodm wants to merge 5 commits intoelastic:mainfrom
mrodm:update_go_1.24.7
Closed

Update go runtime 1.24.7#1425
mrodm wants to merge 5 commits intoelastic:mainfrom
mrodm:update_go_1.24.7

Conversation

@mrodm
Copy link
Copy Markdown
Contributor

@mrodm mrodm commented Sep 11, 2025

This PR updates the Go runtime to 1.24.7.
More information at https://go.dev/doc/devel/release#go1.24.0

Added workaround in Windows step to run unit tests to be able to set Golang 1.24.7.
In Chocolatey, Golang 1.24.7 is not available because there is already a 1.25.0 version.
Version history:
https://community.chocolatey.org/packages/golang#versionhistory

choloatey golang version history

For that, added a condition to set 1.24.6 in case 1.24.7 comes from .go-version. Even with that, later it is downloaded 1.24.7:
https://buildkite.com/elastic/package-registry/builds/1299#01993952-e2c2-4b16-9a3a-65df842d8aa6/92-122

go: downloading go1.24.7 (windows/amd64)
go version go1.24.7 windows/amd64

@mrodm mrodm self-assigned this Sep 11, 2025
@prodsecmachine
Copy link
Copy Markdown

prodsecmachine commented Sep 11, 2025

🎉 Snyk checks have passed. No issues have been found so far.

security/snyk check is complete. No issues have been found. (View Details)

license/snyk check is complete. No issues have been found. (View Details)

@elasticmachine
Copy link
Copy Markdown

💚 Build Succeeded

History

cc @mrodm

@mrodm mrodm marked this pull request as ready for review September 11, 2025 15:44
@mrodm mrodm requested a review from a team as a code owner September 11, 2025 15:44
@mrodm
Copy link
Copy Markdown
Contributor Author

mrodm commented Sep 12, 2025

Should it be updated to 1.25.1 instead of using the latest 1.24.x version ? @elastic/ecosystem

@jsoriano
Copy link
Copy Markdown
Member

Should it be updated to 1.25.1 instead of using the latest 1.24.x version ? @elastic/ecosystem

I think we could go with 1.25, yes.

1.25 includes also the enhancements in JSON performance, right? We could leverage that.

@mrodm
Copy link
Copy Markdown
Contributor Author

mrodm commented Sep 12, 2025

Should it be updated to 1.25.1 instead of using the latest 1.24.x version ? @elastic/ecosystem

I think we could go with 1.25, yes.

1.25 includes also the enhancements in JSON performance, right? We could leverage that.

Ok, I'll update it to 1.25.x

About JSON performance, I think it has to be enabled an experimental feature for that. But it looks like there could be still some issues: golang/go#74617

@mrodm
Copy link
Copy Markdown
Contributor Author

mrodm commented Sep 15, 2025

Closing in favor of #1426

@mrodm mrodm closed this Sep 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants